Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(privacy): add blacklist #695

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

n1rwana
Copy link
Member

@n1rwana n1rwana commented Aug 27, 2022

Добавляет возможность вносить пользователей в чёрный список. При добавлении пользователя target в чёрный список пользователя author, от пользователя target полностью скрывается страница пользователя author, и весь его контент, такой как фотографии, видео, заметки, список друзей и групп.

Если добавить в ЧС администратора инстанса, на него не будут действовать ограничения, и он по-прежнему будет видеть страницу и контент пользователя.

image
image

Resolve #226

@veselcraft veselcraft requested a review from celestora August 27, 2022 16:16
@celestora
Copy link
Member

@n1rwana Плохая идея

@celestora
Copy link
Member

@n1rwana А нет реализация неплохая, но есть пара замечаний, завтра посмотрю

@ghost
Copy link

ghost commented Aug 27, 2022

Если добавить в ЧС администратора инстанса, на него не будут действовать ограничения, и он по-прежнему будет видеть страницу и контент пользователя.

Зачем?

@n1rwana
Copy link
Member Author

n1rwana commented Aug 27, 2022

Зачем?

Например, если на пользователя отправят жалобу в тех. поддержку, а Агент Поддержки каким-то образом окажется в ЧС. ¯\_(ツ)_/¯

Добавил возможность настроить это в конфиге.

@ghost
Copy link

ghost commented Aug 28, 2022

Агент Поддержки каким-то образом окажется в ЧС

Если на то пошло, то можно главным администраторам показывать всю информацию, а второстепенным только субъект жалобы (то есть посты, сообщения или приложения), но для этого нужно расширить функционал репортов.

@veselcraft
Copy link
Member

Чуть не забыл, а что насчёт API? Если человек в черном списке, то он как-нибудь тоже должен ограничиваться

Web/Models/Repositories/Blacklists.php Outdated Show resolved Hide resolved
Web/Presenters/BlacklistPresenter.php Outdated Show resolved Hide resolved
Web/Presenters/BlacklistPresenter.php Outdated Show resolved Hide resolved
Web/Presenters/VideosPresenter.php Outdated Show resolved Hide resolved
Copy link
Member

@veselcraft veselcraft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну тут пару изменений 👉👈

Web/Models/Entities/User.php Outdated Show resolved Hide resolved
Web/Presenters/BlacklistPresenter.php Outdated Show resolved Hide resolved
Web/Presenters/NotesPresenter.php Outdated Show resolved Hide resolved
Web/Presenters/PhotosPresenter.php Outdated Show resolved Hide resolved
Web/Presenters/PhotosPresenter.php Outdated Show resolved Hide resolved
Web/Presenters/UserPresenter.php Outdated Show resolved Hide resolved
Web/Presenters/VideosPresenter.php Outdated Show resolved Hide resolved
Web/Presenters/templates/User/Settings.xml Outdated Show resolved Hide resolved
Web/Presenters/templates/User/View.xml Outdated Show resolved Hide resolved
Web/di.yml Outdated Show resolved Hide resolved
Copy link
Member

@veselcraft veselcraft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Всё гуд, но есть две переведённые строки и nl

VKAPI/Handlers/Users.php Show resolved Hide resolved
Web/Presenters/BlacklistPresenter.php Outdated Show resolved Hide resolved
Web/Presenters/BlacklistPresenter.php Outdated Show resolved Hide resolved
locales/ru.strings Outdated Show resolved Hide resolved
locales/en.strings Outdated Show resolved Hide resolved
@n1rwana n1rwana requested a review from celestora September 5, 2022 19:30
@veselcraft
Copy link
Member

@celestora сестра проверь пулл пж 👉👈

@veselcraft
Copy link
Member

актуально?

@WerySkok
Copy link
Member

Ого я даже и не подозревал про существование сего замечательного пр

@Lumaeris
Copy link
Member

Актуально? @celestora проверь

@celestora celestora force-pushed the master branch 3 times, most recently from a9d021b to cbec4b5 Compare February 7, 2023 20:46
@NoPlagiarism
Copy link
Contributor

@n1rwana время обнов

@veselcraft
Copy link
Member

@n1rwana можешь свою ветку в нашу организацию перебросить? я допилю скорее всего

@openvk-org openvk-org force-pushed the master branch 15 times, most recently from e198aff to 2db815d Compare June 15, 2023 12:48
@ghost
Copy link

ghost commented Jun 28, 2023

Стоит ещё учесть, что с появлением поиска, нужно добавить скрытие постов\коментариев пользователя в поиске.

@nbt1248
Copy link

nbt1248 commented Oct 3, 2023

ну че когда

@WerySkok
Copy link
Member

WerySkok commented Oct 3, 2023

Там мерж конфликт

@ghost
Copy link

ghost commented Nov 10, 2023

Мержните (или допилите) PR, пожалуйста, меня этот индивид заеб
image

@veselcraft
Copy link
Member

@JBLHRD этот хуебес нас тоже заебал кстати

@nbt1248
Copy link

nbt1248 commented Dec 11, 2024

НУ ЧЁ КОГДА
ПОЧИНИТЕ УЖЕ СВОИ КОНФЛИКТЫ

@mrilyew
Copy link
Contributor

mrilyew commented Dec 11, 2024

ну ветку в этом репо я обновил а как её здесь обновить то

@mrilyew
Copy link
Contributor

mrilyew commented Dec 11, 2024

Ладно, обновить эту ветку вряд ли удастся, но зацените короч
1
2

@mrilyew mrilyew changed the title Чёрный список feat(privacy): add blacklist Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Доделать
Development

Successfully merging this pull request may close these issues.

Чёрный список
8 participants